Skip to content

Fix audio_cover_strength in reference mode, lm_negative_prompt default, ace-understand binary, and metadata persistence#18

Merged
lmangani merged 2 commits intomainfrom
copilot/fix-ui-audio-cover-strength
Mar 13, 2026
Merged

Fix audio_cover_strength in reference mode, lm_negative_prompt default, ace-understand binary, and metadata persistence#18
lmangani merged 2 commits intomainfrom
copilot/fix-ui-audio-cover-strength

Conversation

Copy link
Contributor

Copilot AI commented Mar 13, 2026

Four distinct bugs in the acestep-cpp UI/backend integration: missing slider in reference mode, wrong LM default, ace-understand never installed to bin/, and understand results not persisted.

UI fixes (components/CreatePanel.tsx)

  • lm_negative_prompt default: 'NO USER INPUT'''
  • audio_cover_strength slider: now shown in the AUDIO: REFERENCE tab when a reference audio is loaded (was cover-only)
  • Understand (✨) button + result panel: added to the Lego tab backing-track player; Reference and Cover tabs already had it
  • handleUnderstand: routes to /:id/understand (persists to DB) when the audio URL matches an uploaded reference track, falls back to /understand-url otherwise

ace-understand binary not found (build.sh, build.bat)

ace-understand was never copied to ./bin/ post-build — only ace-qwen3, dit-vae, and neural-codec were. Config auto-discovery already looked in ./bin/; the copy step was just missing.

# before
for name in ace-qwen3 dit-vae neural-codec; do
# after
for name in ace-qwen3 dit-vae ace-understand neural-codec; do

Also added ACE_UNDERSTAND_BIN to .env.example override docs.

Metadata persistence (server/)

  • Migration (migrate.ts): adds understand_metadata TEXT column to reference_tracks via a safe incremental ALTER TABLE (idempotent, catches already-exists).
  • Route (referenceTrack.ts): POST /:id/understand now saves the result JSON to understand_metadata after analysis — best-effort, non-fatal on save failure.

Reference audio in spawn mode (server/src/services/acestep.ts)

When referenceAudioUrl is set in text-to-music mode, reference_audio and audio_cover_strength are now written into the request JSON passed to the binary — mirrors what the HTTP API path already did.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ompt default, ace-understand binary, metadata saving

Co-authored-by: lmangani <1423657+lmangani@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix UI issues for acestep-cpp integration Fix audio_cover_strength in reference mode, lm_negative_prompt default, ace-understand binary, and metadata persistence Mar 13, 2026
Copilot AI requested a review from lmangani March 13, 2026 21:20
@lmangani lmangani marked this pull request as ready for review March 13, 2026 22:28
@lmangani lmangani merged commit 6e1d10b into main Mar 13, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants